Description

Simplified capture querying for Verification tool to address issues seen when updating existing tags (e.g., species) on already approved captures.

Issue(s) addressed

• Resolves Verify: Captures do not update correctly following approval #230

What kind of change(s) does this PR introduce?

• Enhancement
• Bug fix
• Refactor

Please check if the PR fulfils these requirements

• The commit message follows our guidelines
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

Issue

What is the current behavior?

• Verify tool attempts to cache captures and remove them one by one as they are approved. This assumes that captures will no longer be included in the filter and was designed for when the tool was only used to verify captures.
• Even when captures should be removed from the list, the captureImages state is set in VerifyContext in a synchronous loop and subsequently referenced before it can be guaranteed to have been updated, causing unexpected behaviour.
• The result is that re-tagged captures are not removed from the page when they should be.

What is the new behavior?

• When batch approval is complete, the captures query is rerun to ensure the user sees the captures that match the filter.

Breaking change

Does this PR introduce a breaking change?
No

Other useful information

• Removed all the unused, commented-out code
• Moved captures query effect to the context where it belongs
• Updated the options for number of captures per page following user feedback